Skip to content
This repository was archived by the owner on Jan 16, 2023. It is now read-only.

Add demo example#19

Open
ilyamore88 wants to merge 8 commits intomasterfrom
add-demo-example
Open

Add demo example#19
ilyamore88 wants to merge 8 commits intomasterfrom
add-demo-example

Conversation

@ilyamore88
Copy link
Copy Markdown
Member

No description provided.

background: var(--color-bg-element);
box-shadow: 0 1px 1px 0 rgba(0, 0, 0, 0.05);
padding: 8px 15px;
font-family: "Roboto";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пропиши лучше один раз в стилях к html

border-radius: 3px;
}

&::-moz-range-track {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

общие стили надо вынести (перечислить селекторы через запятую)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В хроме есть баг, который не позволяет этого сделать. Если писать селекторы через зяпятую, то тогда стили для кастомных селекторов не применяются.

@ilyamore88 ilyamore88 marked this pull request as ready for review November 11, 2019 18:26
@@ -0,0 +1 @@

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

-moz-osx-font-smoothing: grayscale;
}

*, *:before, *:after {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
*, *:before, *:after {
*, *::before, *::after {

@@ -0,0 +1,28 @@
@import "variables.css";

@import url('https://fonts.googleapis.com/css?family=Roboto&display=swap');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

плохо загружать шрифты таким образом. Надо в head прописать

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants